-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Do not merge] Test/sitealert banner #693
base: develop
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for ghg-demo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@dzole0311 the preview is missing the icons for the banners for some reason 🤔 |
@slesaad looking into it |
@slesaad @paridhi-parajuli I found the issue and opened a PR with a fix and tested the change here in the GHG Center: #698 It was a front-end pipeline problem since one of the USWDS-provided utils was not working as expected, and we will need a veda-ui hotfix release 😞 Depending on how urgent this is, an alternative is to manually copy the few missing icons into the static directory and link them in the config file, which I could do for you. |
@dzole0311 thanks for the fix! no rush right now, we can pull it in from the next release! |
This PR tests the addition banner, site alerts and cookie consent form to GHGC portal.
Reference : https://github.com/NASA-IMPACT/veda-ui/blob/3e9ce51e909b2eea78d4c108e8c37a9e49374ef8/mock/veda.config.js
It requires the the following env vars to be set in the .env file
COOKIE_CONSENT_FORM = 'TRUE'